From: Michele Gaio <micxgx@gmail.com>
Date: Wed, 4 Nov 2015 00:56:16 +0000 (+0000)
Subject: AppIconMenu: properly destroy on source destroy.
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~855
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=1545596c7ec53e16dc6bbc5261c9f8907d291e2a;p=gnome-shell.git

AppIconMenu: properly destroy on source destroy.

If the source actor is destroyed while the popupMenu is shown -- this
can happen if a non favorite application was closing or crashes -- the
menu actor is improperly destroyed.

This makes the popupMenu close first and does a clean ungrab instead.

https://bugzilla.gnome.org/show_bug.cgi?id=757556
---

diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 58adf3a32..cae175709 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -1814,7 +1814,7 @@ const AppIconMenu = new Lang.Class({
             if (!source.actor.mapped)
                 this.close();
         }));
-        source.actor.connect('destroy', Lang.bind(this, function () { this.actor.destroy(); }));
+        source.actor.connect('destroy', Lang.bind(this, this.destroy));
 
         Main.uiGroup.add_actor(this.actor);
     },